Drop bin/cua, fold DESIGN.md into architecture.md, delete .cursor - #27
Merged
Conversation
- Remove bin/cua wrapper and update install paths in README.md and packages/cli/README.md to run the CLI via npx tsx from source. The wrapper assumed a built dist/cli.js that's currently broken at runtime in the workspace. - Delete .cursor/ directory (stale plans/cua-ai-agent.md only). - Merge docs/DESIGN.md into docs/architecture.md as a new "Product principles" section covering "Package the boring plumbing", "Do not over-own the agent", and "Keep model refs explicit"; drop the layering diagram and "Current Surface" section (already covered in architecture.md). Delete docs/DESIGN.md. - Mark docs/cua-cli-harness-migration.md status completed and drop the npm deprecate follow-up — the deprecated packages were never published.
- Swap the README cua() shell function to a cwd-preserving form so --out, transcript bucketing, and .agents/skills discovery use the caller's directory, not the repo root. - Add a brief "from the repo root" note to the cli README install block to remove path ambiguity. - Drop the stale "Out of scope: npm deprecate of the published provider packages" bullet from the PR 4 section in docs/cua-cli-harness-migration.md — those packages were never published to npm. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
rgarcia
marked this pull request as ready for review
June 12, 2026 15:15
|
Firetiger deploy monitoring skipped This PR didn't match the auto-monitor filter configured on your GitHub connection:
Reason: PR appears to be in the To monitor this PR anyway, reply with |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Repo cleanup follow-up to the cua-cli → CuaAgentHarness migration:
bin/cuawrapper script and update install instructions inREADME.mdandpackages/cli/README.mdto run the CLI vianpx tsx packages/cli/src/cli.ts. The wrapper exec'dnode packages/cli/dist/cli.js,which doesn't currently work because the dist build emits relative
imports without
.jsextensions (a separate pre-existing build issue).Verified the
npx tsxpath works from this checkout..cursor/entirely; it only held a staleplans/cua-ai-agent.mdand was already gitignored.docs/DESIGN.mdintodocs/architecture.mdas a new"Product principles" section covering "Package the boring plumbing",
"Do not over-own the agent", and "Keep model refs explicit". Dropped
the layering diagram and "Current Surface" section (both already
covered in architecture.md). Deleted
docs/DESIGN.md. No other filesin the repo referenced DESIGN.md.
docs/cua-cli-harness-migration.md:status set to completed; removed the "npm deprecate" follow-up since
the deprecated provider packages were never published to npm.
Tests
npm test --workspace @onkernel/cua-cli— 29 passed, 4 skipped(ptywright fixture tests skip without native binding, as expected
outside CI).
npm run build --workspace @onkernel/cua-ai,npm run build --workspace @onkernel/cua-agent,npm run build --workspace @onkernel/cua-cli— all green.npm run typecheck— green.npm run buildstill hits a pre-existing@onkernel/ptywrightnative combine-archives failure unrelated to this PR (CI uses the
zig-cache to avoid it).
Judgment calls
npx tsxovernpm linkbecausenpm linkwould symlink the bin entry to the samedist/cli.jsthatfails at runtime.
npx tsxreads the TypeScript source directly and isverified working in this checkout.
Note
Low Risk
Documentation and dev-workflow changes only; no runtime or application logic is modified.
Overview
Removes the
bin/cuawrapper and documents running the CLI withnpx tsx packages/cli/src/cli.ts(plus an optional shell function that preserves the caller’s cwd). Dropsnpm run buildfrom the quickstart/install path because the old wrapper targeteddist/cli.js, which doesn’t run cleanly with relative imports lacking.jsextensions.Deletes
.cursor/(stale plan only) anddocs/DESIGN.md, folding its product principles intodocs/architecture.mdas a new section—without the old layering diagram or “Current Surface” block already covered elsewhere.Updates
docs/cua-cli-harness-migration.md: migration status completed, and removesnpm deprecatefollow-ups for packages that were never published.Reviewed by Cursor Bugbot for commit bc05db4. Bugbot is set up for automated code reviews on this repo. Configure here.